Skip to content

expand ~ in allowed paths#5

Merged
jph00 merged 1 commit into
mainfrom
expanduser
Jun 2, 2026
Merged

expand ~ in allowed paths#5
jph00 merged 1 commit into
mainfrom
expanduser

Conversation

@kafkasl
Copy link
Copy Markdown
Contributor

@kafkasl kafkasl commented Apr 30, 2026

Ok paths need to expand the user otherwise we get the error below:

image

I added a small test and made sure it failed w/o the fix

@kafkasl kafkasl requested a review from jph00 April 30, 2026 10:50
@kafkasl kafkasl self-assigned this Apr 30, 2026
@kafkasl kafkasl added the bug Something isn't working label Apr 30, 2026
@jph00
Copy link
Copy Markdown
Contributor

jph00 commented Jun 2, 2026

Sorry I missed this one!

@jph00 jph00 merged commit e588a45 into main Jun 2, 2026
@jph00 jph00 added enhancement New feature or request and removed bug Something isn't working labels Jun 2, 2026
@jph00 jph00 changed the title fixed bug where we do not expand ~ in allowed paths expand ~ in allowed paths Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants